Attempt to fix problem with Nexus getting stuck while running with too little memory #4189
Conversation
…e-Science/AzureTRE into increase-nexus-java-memory
…e-Science/AzureTRE into increase-nexus-java-memory
|
@microsoft-github-policy-service agree [company="Barts Health NHS Trust"]
************************************************************************************** ******************************
This message may contain confidential information. If you are not the intended recipient please:
i) inform the sender that you have received the message in error before deleting it; and
ii) do not disclose, copy or distribute information in this e-mail or take any action in relation to its content (to do so is strictly prohibited and may be unlawful).
Thank you for your co-operation.
NHSmail is the secure email, collaboration and directory service available for all NHS staff in England. NHSmail is approved for exchanging patient data and other sensitive information with NHSmail and other accredited email services.
For more information and to find out how you can switch visit Joining NHSmail - NHSmail Support<https://support.nhs.net/article-categories/joining-nhsmail/>
|
|
Thanks @TonyWildish will have a look. Any chance you can add a line to the |
done... |
|
Just noting this before review, had an issue with a nexus instance due to: I wonder if we should avoid the B series. |
|
I think a D2v3 is likely to resolve the issue. Also worth comparing price of Standard_B8ms and D8v3. Will try have a look later. |
templates/shared_services/sonatype-nexus-vm/scripts/deploy_nexus_container.sh
Show resolved
Hide resolved
|
I've had another one of these die today, not sure if the nexus container is behaving differently due to an update. @TonyWildish-BH let me know your thoughts on the changes, but keen to get this in, then follow with an option to select VM size. Thanks. |
Unit Test Results0 tests 0 ✅ 0s ⏱️ Results for commit 6be2a7e. ♻️ This comment has been updated with latest results. |
|
@TonyWildish Can you resolve the conflicts, then we can get this merged in. |
Co-authored-by: Marcus Robinson <[email protected]>
|
changes not covered in tests |
|
/test-force-approve |
|
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit 6be2a7e) (in response to this comment from @tim-allen-ck) |
Attempts to resolve #4074
What is being addressed
The current Nexus VM is too small, and nexus doesn't get enough memory to run properly. This causes it to wedge frequently.
How is this addressed
Standard_B2stoStandard_B8ms